Skip to content

[Issue #1068] investigate issues with Retina under high concurrency #1069

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 22 commits into from
Oct 16, 2025
Merged

[Issue #1068] investigate issues with Retina under high concurrency #1069

merged 22 commits into from
Oct 16, 2025

Conversation

@gengdy1545
Copy link
Collaborator

@gengdy1545 gengdy1545 commented Sep 29, 2025 •
edited
Loading

The following tests have been added:

  1. Retina overall concurrency testing
  2. Index concurrency testing under actual load

@gengdy1545 gengdy1545 marked this pull request as draft September 30, 2025 03:02
@bianhq bianhq added the bug Something isn't working label Sep 30, 2025
@bianhq bianhq added this to the Real-time CRUD milestone Sep 30, 2025
@gengdy1545 gengdy1545 force-pushed the chore/updateRetinaTest branch from 61c7967 to 1293f65 Compare October 13, 2025 03:36
@gengdy1545 gengdy1545 marked this pull request as ready for review October 13, 2025 03:44
@gengdy1545 gengdy1545 requested review from AntiO2 and bianhq October 14, 2025 14:27
@gengdy1545
fix: pass in raw bytes directly when testing addRow
2cf0698
@gengdy1545
feat: add steam update test
1a8f1a2
@gengdy1545
fix: values.length is already checked in addRow
22c1f19
@gengdy1545
feat: update retina service test
8a4fea1
@gengdy1545
feat: update retina service test
115c44a
@gengdy1545
feat: update retina service test
82b5153
@gengdy1545
feat: update retina service test
03093e3
@gengdy1545
feat: update retina pom
87d695d
@gengdy1545
feat: update retina test
c29d8b9
@gengdy1545
feat: update retina pom
09a68ad
@gengdy1545
feat: add retina trace test for index service
f209303
@gengdy1545
feat: update retina trace test for index service
d50c3e9
@gengdy1545
feat: update retina trace test for index service
b7498d4
@gengdy1545
feat: reduce file flushing time to 30 seconds
266b7ab
@gengdy1545
fix: add the index-memory dependency
e8207ac
@gengdy1545
feat: update test for retina service
9f47f18
@gengdy1545
feat: update test for retina service
eb56e53
@gengdy1545
feat: avoid hard-coding the keyColumn for SinglePointIndex
9805b9b
@gengdy1545
fix: use sharding to enable parallel processing
16b4107
AntiO2
AntiO2 reviewed Oct 15, 2025
View reviewed changes
@gengdy1545 gengdy1545 force-pushed the chore/updateRetinaTest branch from e8ae9c5 to 44da852 Compare October 15, 2025 14:03
bianhq
bianhq approved these changes Oct 16, 2025
View reviewed changes
Copy link
Contributor

@bianhq bianhq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@bianhq bianhq merged commit 162f975 into pixelsdb:master Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AntiO2 AntiO2 AntiO2 left review comments

@bianhq bianhq bianhq approved these changes

Assignees

@bianhq bianhq

@AntiO2 AntiO2

@gengdy1545 gengdy1545

Labels

bug Something isn't working

Projects

None yet

Milestone

Real-time CRUD

Development

Successfully merging this pull request may close these issues.

3 participants

@gengdy1545 @bianhq @AntiO2